Skip to content

feat: add uptime monitors with datadog synthetics#587

Closed
JacobCoffee wants to merge 3 commits intomainfrom
uptime-monitors
Closed

feat: add uptime monitors with datadog synthetics#587
JacobCoffee wants to merge 3 commits intomainfrom
uptime-monitors

Conversation

@JacobCoffee
Copy link
Copy Markdown
Member

Description

@JacobCoffee JacobCoffee requested a review from ewdurbin as a code owner June 10, 2025 18:44
Comment thread salt/top.sls
- rsyslog
- datadog
- datadog.synthetics
- secrets.datadog
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this the right place for this? it felt wrong but it needed them to be available

Comment thread salt/datadog/synthetics.sls
Comment thread salt/datadog/synthetics.sls
@ewdurbin
Copy link
Copy Markdown
Member

-1 overall. I don't think I really want to drive our configuration for synthetics through salt.

Maybe time to start a terraform repo for non-server things: https://registry.terraform.io/providers/DataDog/datadog/3.33.0/docs/resources/synthetics_test

@JacobCoffee
Copy link
Copy Markdown
Member Author

for posterity we did this instead: https://github.com/psf/infra/tree/main/terraform/synthetics

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants